Conversation
WalkthroughОбновление системы Emag включает изменение типа по умолчанию компонента с Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Suggested labels
Suggested reviewers
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
Resources/Locale/ru-RU/WWDP_TRANSLATION/prototypes/entities/objects/tools/emag.ftl (2)
9-10: Рассмотрите уточнение описания.Описание упоминает только "электронные замки", но согласно целям PR, дешифратор доступа может открывать как шлюзы (airlocks), так и замки (locks). Рекомендую добавить уточнение для полноты.
🔎 Предлагаемое уточнение описания
- .desc = Перепрограммированный конфигуратор доступа, способный разблокировать электронные замки. + .desc = Перепрограммированный конфигуратор доступа, способный разблокировать шлюзы и электронные замки.
11-11: Обновите суффикс для точности.Суффикс "Емаг, Только Шлюзы" указывает, что устройство работает только со шлюзами, но согласно целям PR, дешифратор доступа может открывать как шлюзы, так и замки. Рекомендую обновить суффикс для более точного отражения функциональности.
🔎 Предлагаемое обновление суффикса
- .suffix = Емаг, Только Шлюзы + .suffix = Емаг, Шлюзы и Замки
📜 Review details
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (8)
Content.Shared/Emag/Components/EmagComponent.csContent.Shared/Emag/Systems/EmagSystem.csResources/Locale/ru-RU/WWDP_TRANSLATION/prototypes/entities/objects/tools/emag.ftlResources/Locale/ru-RU/deltav/store/uplink-catalog.ftlResources/Locale/ru-RU/emag/emag.ftlResources/Locale/ru-RU/medical/components/cloning-pod-component.ftlResources/Locale/ru-RU/store/uplink-catalog.ftlResources/Prototypes/Catalog/uplink_catalog.yml
🧰 Additional context used
🧠 Learnings (2)
📓 Common learnings
Learnt from: Vaaankas
Repo: WWhiteDreamProject/wwdpublic PR: 269
File: Content.Server/_White/Examine/ExaminableCharacterSystem.cs:0-0
Timestamp: 2025-03-03T15:02:14.394Z
Learning: User Vaaankas prefers communication in Russian language. Respond to them in Russian rather than English.
📚 Learning: 2025-10-22T20:13:31.389Z
Learnt from: Lachklen
Repo: WWhiteDreamProject/wwdpublic PR: 863
File: Resources/Locale/en-US/_white/cards/hobby.ftl:5-5
Timestamp: 2025-10-22T20:13:31.389Z
Learning: В PR по EvacPod для репозитория WWhiteDreamProject/wwdpublic: если опечатка в ключе локализации (например, card-ep_hobby_mashrooms) не влияет на отображение и используется последовательно, автор предпочитает не блокировать PR; вместо этого предлагать необязательный follow-up issue.
Applied to files:
Resources/Locale/ru-RU/medical/components/cloning-pod-component.ftl
🧬 Code graph analysis (1)
Content.Shared/Emag/Systems/EmagSystem.cs (1)
Content.Shared/Emag/Components/EmagComponent.cs (1)
Access(11-36)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (5)
- GitHub Check: Test Packaging
- GitHub Check: YAML Linter
- GitHub Check: YAML map schema validator
- GitHub Check: build (ubuntu-latest)
- GitHub Check: build (ubuntu-latest)
🔇 Additional comments (8)
Resources/Locale/ru-RU/emag/emag.ftl (1)
1-1: Изменение терминологии выглядит корректно.Замена "Карточка" на "Устройство" делает локализацию более универсальной и соответствует восстановлению полной функциональности emag согласно целям PR.
Resources/Locale/ru-RU/medical/components/cloning-pod-component.ftl (1)
5-5: Консистентное обновление терминологии.Изменение соответствует обновлению в других файлах локализации и поддерживает единообразие терминологии.
Resources/Prototypes/Catalog/uplink_catalog.yml (2)
1185-1191: Обновление стоимости и категории скидки для UplinkAccessBreaker выглядит корректно.Изменения соответствуют целям PR:
- Стоимость изменена с 5 TC на 4 TC ✓
- Категория скидки изменена с
rareDiscountsнаusualDiscounts, что логично при снижении цены
1202-1204: Обновление стоимости для UplinkEmag выглядит корректно.Изменения соответствуют целям PR:
- Стоимость увеличена с 4 TC до 8 TC ✓
- Минимальная цена со скидкой увеличена с 1 TC до 2 TC, что пропорционально увеличению базовой стоимости
Resources/Locale/ru-RU/store/uplink-catalog.ftl (1)
160-160: Обновление описания emag выглядит корректно.Добавление упоминания о возможности "открывать электронные замки" соответствует цели PR по восстановлению полной функциональности emag. Описание теперь более полное и точно отражает возможности устройства.
Resources/Locale/ru-RU/deltav/store/uplink-catalog.ftl (1)
19-20: Переименование ключей локализации выполнено корректно.Старые ключи
uplink-doorjack-*полностью удалены из кодовой базы и успешно заменены наuplink-access-breaker-*. Проверка показала отсутствие ссылок на старые ключи в файлах локализации. Изменения соответствуют целям PR по переименованию.Content.Shared/Emag/Systems/EmagSystem.cs (1)
134-135: Корректное расширение флагового enum.Добавление нового флага
All = Interaction | Accessвыполнено правильно:
- Используется корректный синтаксис битовой комбинации для флагового enum
- Добавлена завершающая запятая на строке 134 для упрощения будущих дополнений
- Существующие битовые операции в методах
CompareFlag,CheckFlagиOnAccessOverriderAccessUpdatedкорректно работают с новым комбинированным флагомContent.Shared/Emag/Components/EmagComponent.cs (1)
28-28: Изменение дефолтного значения расширяет функциональность emag, и все существующие прототипы совместимы.Изменение дефолтного значения с
EmagType.InteractionнаEmagType.Allкорректно. Основной инструмент emag вemag.ymlне указывает явное значениеemagTypeи теперь получит полную функциональность (Interaction + Access). Все остальные устройства с явно заданнымemagType(access_breaker.yml, fax_machine.yml, abductor tools) остаются без изменений и совместимы с новым дефолтным поведением. Изменение не вызывает конфликтов и соответствует целям восстановления полной функциональности emag.
|
Имеет мало смысла, зачем опять баффать емаг, если его не так давно нерфали по определённым причинам? |
|
Видел, что ты написал в чате, отвечу всё же тут. Проблема короче "классики" в том, что не имеет особо много смысла цепляться и держаться за неё. Тут следует разобрать "емаг" как предмет для тритора именно функционально, а не потому, что он был "базой" тритора на протяжении тысячи сотен лет. Емаг "взламывает", взламывает он двери и взламывает от штуки. Короче, тут надо уже крутится вокруг баланса вещей и баланса тритора в принципе, я просто не вижу смысла возвращать емагу прошлый функционал просто потому, что "оно так было". Я бы хотел большей гибкости триторам и больше реакционности на события - они тогда станут веселее и с ними будет сложнее разобраться, давая этакую прослойку в случае ошибки игрока за тритора, которая не дана обычным игрокам. |
|
ванкс опять срет своими "замечательными" идеями. |
3 participants
🆑 vanx
tweak Стоимость дешифратора: 5 -> 4 ТК.